Skip to content

feat: dependency changes to clean up unused resources#227

Open
Tejasri-Microsoft wants to merge 9 commits into
devfrom
psl-ts-cleanup
Open

feat: dependency changes to clean up unused resources#227
Tejasri-Microsoft wants to merge 9 commits into
devfrom
psl-ts-cleanup

Conversation

@Tejasri-Microsoft
Copy link
Copy Markdown
Contributor

@Tejasri-Microsoft Tejasri-Microsoft commented May 6, 2026 •
edited
Loading

This pull request primarily removes unused resources and dependencies to streamline the infrastructure and backend API configuration. The main changes include deleting unused storage containers and Cosmos DB containers, as well as cleaning up Python dependencies.

Infrastructure cleanup:

  • Removed the data blob container from the storageAccount module in both infra/main.bicep and infra/main_custom.bicep, leaving the containers list empty. [1] [2]
  • Removed the process_statuses container from the Cosmos DB configuration in both infra/main.bicep and infra/main_custom.bicep. [1] [2]

Backend dependency cleanup:

  • Removed the azure-search-documents package from the src/backend-api/pyproject.toml dependencies, as it is no longer needed.
  • Updated the semantic-kernel dependency to remove the [azure] extra, simplifying the requirement to just semantic-kernel==1.40.0.This pull request makes several infrastructure and dependency changes to clean up unused resources and dependencies. The main focus is on removing unused storage containers, Cosmos DB containers, and a Python package.

Infrastructure cleanup:

  • Removed the data container from the blobServices.containers list in the storageAccount module in infra/main.bicep, resulting in no containers being created by default.
  • Removed the process_statuses container from the Cosmos DB containers list in the cosmosDb module in infra/main.bicep.

Dependency cleanup:

  • Removed the azure-search-documents package from the Python backend API dependencies in src/backend-api/pyproject.toml.## Purpose
  • ...

Does this introduce a breaking change?

  • Yes
  • [ x] No

Golden Path Validation

  • [x ] I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • [x ] I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

Tejasri-Microsoft and others added 2 commits May 6, 2026 20:14
Copilot AI review requested due to automatic review settings May 6, 2026 15:36
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026 •
edited
Loading

Coverage

Coverage Report •
FileStmtsMissCoverMissing
TOTAL274024789% 
report-only-changed-files is enabled. No files were changed during this commit :)

Tests Skipped Failures Errors Time
44 0 💤 0 ❌ 0 🔥 14.464s ⏱️

@Tejasri-Microsoft Tejasri-Microsoft changed the title feat:dependency changes to clean up unused resources feat: dependency changes to clean up unused resources May 6, 2026
@Tejasri-Microsoft Tejasri-Microsoft review requested due to automatic review settings May 7, 2026 05:18
@Tejasri-Microsoft
Merge branch 'dev' of https://github.com/microsoft/Container-Migratio…
489604b 
…n-Solution-Accelerator into psl-ts-cleanup

Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings May 8, 2026 06:56
Copilot AI reviewed May 8, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

Comment thread src/backend-api/pyproject.toml
Comment thread src/backend-api/pyproject.toml
Copilot AI review requested due to automatic review settings May 8, 2026 07:02
@Tejasri-Microsoft Tejasri-Microsoft requested a review from Copilot May 8, 2026 08:12
Copilot AI reviewed May 8, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

Comment thread src/backend-api/pyproject.toml
Comment thread src/backend-api/pyproject.toml
@Tejasri-Microsoft Tejasri-Microsoft requested a review from Copilot May 8, 2026 08:27
Copilot AI reviewed May 8, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 5 comments.

Comment thread src/backend-api/pyproject.toml
Comment thread infra/main.bicep
Comment thread infra/main.bicep
Comment thread infra/main_custom.bicep
Comment thread infra/main_custom.bicep
@Tejasri-Microsoft
commit
17338b8 
Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings May 12, 2026 08:55
Copilot AI reviewed May 12, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 6 out of 8 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • src/frontend/package-lock.json: Language not supported

Comment thread src/frontend/src/pages/batchView.tsx Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@Avijit-Microsoft Avijit-Microsoft Awaiting requested review from Avijit-Microsoft Avijit-Microsoft is a code owner

@Roopan-Microsoft Roopan-Microsoft Awaiting requested review from Roopan-Microsoft Roopan-Microsoft is a code owner

@Prajwal-Microsoft Prajwal-Microsoft Awaiting requested review from Prajwal-Microsoft Prajwal-Microsoft is a code owner

@Vinay-Microsoft Vinay-Microsoft Awaiting requested review from Vinay-Microsoft Vinay-Microsoft is a code owner

@aniaroramsft aniaroramsft Awaiting requested review from aniaroramsft aniaroramsft is a code owner

@Dongbumlee Dongbumlee Awaiting requested review from Dongbumlee Dongbumlee is a code owner

@sethsteenken sethsteenken Awaiting requested review from sethsteenken sethsteenken is a code owner

@toherman-msft toherman-msft Awaiting requested review from toherman-msft toherman-msft is a code owner

@nchandhi nchandhi Awaiting requested review from nchandhi nchandhi is a code owner

@dgp10801 dgp10801 Awaiting requested review from dgp10801 dgp10801 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Tejasri-Microsoft